Skip to content

[Brand Refactor] Small fixes batch - #1387

Merged
rezrah merged 12 commits into
mainfrom
danielguillan/brand-refactor-p0-fixes-batch
Jul 3, 2026
Merged

[Brand Refactor] Small fixes batch#1387
rezrah merged 12 commits into
mainfrom
danielguillan/brand-refactor-p0-fixes-batch

Conversation

@danielguillan

@danielguillan danielguillan commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Towards https://github.com/github/brand-experience/issues/289

Fixes 4 reported UI bugs.

Resolves https://github.com/orgs/github/projects/23995/views/5?pane=issue&itemId=185434044
Resolves https://github.com/github/brand-experience/issues/256
Resolves https://github.com/orgs/github/projects/23995/views/5?pane=issue&itemId=196640953
Resolves https://github.com/orgs/github/projects/23995/views/5?pane=issue&itemId=196639023

List of notable changes:

  • Updates Pillar icons with background border-radius to 12px
  • Updates Tiles layout to use three columns on tablet viewports
  • Fixes Statistic social proof text color consistency
  • Adjusts MediaPlaylist spacing on narrow viewports

What should reviewers focus on?

  • Check the story links below.
  • Check we're comfortable with the new VideoPlayer bottom-controls API and the responsive spacing/column changes.

Steps to test:

Use the following links to verify changes:

  1. Pillar icon radius
  2. Tiles tablet layout
  3. Statistic text color
  4. MediaPlaylist narrow spacing

Contributor checklist:

  • All new and existing CI checks pass
  • Tests prove that the feature works and covers both happy and unhappy paths
  • Any drop in coverage, breaking changes or regressions have been documented above
  • UI Changes contain new visual snapshots (generated by adding update snapshots label to the PR)
  • All developer debugging and non-functional logging has been removed
  • Related issues have been referenced in the PR description

Reviewer checklist:

  • Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
  • Check that tests prove the feature works and covers both happy and unhappy paths
  • Check that there aren't other open Pull Requests for the same update/change

Screenshots:

Please try to provide before and after screenshots or videos

Before After
image image
image image

@changeset-bot

changeset-bot Bot commented Jul 1, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ead6d62

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@primer/react-brand Patch
@primer/brand-docs Patch
@primer/brand-css Patch
@primer/brand-primitives Patch
@primer/brand-e2e Patch
@primer/brand-fonts Patch
@primer/brand-mcp Patch
@primer/brand-config Patch
@primer/brand-storybook Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🟢 No design token changes found

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🟢 No unit test coverage changes found

All components and hooks with tests maintain the same coverage as the main branch.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🟢 Bundle size report

CheckMainBranchChange
UMD — full bundle (JS)101.33 kB101.33 kB🟢 No change
UMD — full bundle (CSS)66.74 kB66.82 kB⬆️ +82 B (+0.1%)
ESM — full bundle (JS + CSS)1.51 MB1.51 MB⬆️ +31 B (+0.0%)
ESM — tree-shaken simple (Button)69.69 kB69.70 kB⬆️ +7 B (+0.0%)
ESM — tree-shaken complex (ActionMenu)78.47 kB78.48 kB⬆️ +7 B (+0.0%)

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🟢 No visual differences found

Our visual comparison tests did not find any differences in the UI.

@danielguillan
danielguillan marked this pull request as ready for review July 1, 2026 09:42
@danielguillan
danielguillan requested a review from a team as a code owner July 1, 2026 09:42
Copilot AI review requested due to automatic review settings July 1, 2026 09:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR batches several small UI/UX fixes across Primer Brand components and documentation, including an update to the VideoPlayer API to support bottom-positioned controls and improved autoplay story behavior.

Changes:

  • Extend VideoPlayer with controlsPosition and showControlsWhenPaused, update styles/stories/docs, and adjust useVideo volume/mute state syncing.
  • Update responsive layout/spacing behaviors for Tiles and MediaPlaylist, plus a small Pillar icon-radius tweak.
  • Fix a Statistic example’s text color to be consistent, and add changesets for release notes.
Show a summary per file
File Description
packages/react/src/VideoPlayer/VideoPlayer.tsx Adds new props and modifies control/overlay rendering and positioning logic.
packages/react/src/VideoPlayer/VideoPlayer.test.tsx Adds coverage for new paused-controls/autoplay behaviors, mute sync, and bottom controls positioning.
packages/react/src/VideoPlayer/VideoPlayer.stories.tsx Exposes new props in Storybook controls/default args.
packages/react/src/VideoPlayer/VideoPlayer.module.css.d.ts Updates CSS module typings for new modifier classes.
packages/react/src/VideoPlayer/VideoPlayer.module.css Adds styles for bottom-positioned controls and container positioning.
packages/react/src/VideoPlayer/VideoPlayer.features.stories.tsx Adds an autoplay + bottom-controls feature story.
packages/react/src/VideoPlayer/hooks/useVideo.tsx Syncs volume/mute state from the underlying <video> element more reliably.
packages/react/src/Tiles/Tiles.module.css Adjusts tablet viewport behavior to use 3 columns.
packages/react/src/Statistic/Statistic.examples.stories.tsx Updates social proof example heading color token usage.
packages/react/src/Pillar/Pillar.module.css Updates icon border-radius for “with background” variant.
packages/react/src/MediaPlaylist/MediaPlaylist.module.css Tweaks spacing on narrow viewports and heading/paragraph spacing.
packages/e2e/scripts/playwright/playwright.generate-tests.ts Skips the new autoplay story to avoid flakey visual tests.
apps/next-docs/content/components/VideoPlayer/index.mdx Documents the new autoplay + bottom-controls usage and prop table update.
.changeset/video-player-autoplay-controls.md Changeset entry for VideoPlayer update.
.changeset/tiles-tablet-columns.md Changeset entry for Tiles tablet layout update.
.changeset/statistic-color-consistency.md Changeset entry for Statistic example color update.
.changeset/pillar-icon-radius.md Changeset entry for Pillar icon radius update.
.changeset/media-playlist-narrow-spacing.md Changeset entry for MediaPlaylist spacing update.

Review details

  • Files reviewed: 17/20 changed files
  • Comments generated: 1
  • Review effort level: Low

Comment thread packages/react/src/VideoPlayer/VideoPlayer.tsx Outdated
@rezrah
rezrah temporarily deployed to github-pages July 3, 2026 10:48 — with GitHub Actions Inactive
@rezrah
rezrah merged commit af3ac05 into main Jul 3, 2026
19 of 21 checks passed
@rezrah
rezrah deleted the danielguillan/brand-refactor-p0-fixes-batch branch July 3, 2026 10:49
@primer primer Bot mentioned this pull request Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants